Open
Conversation
Dokploy Preview Deployment
|
PR Preview BuildsBuild Number: 615 Deploy
|
Contributor
There was a problem hiding this comment.
Pull request overview
Adds an NTUT Portal academic calendar integration by introducing a new CalendarService that fetches events from calModeApp.do, wiring it into existing Riverpod DI and updating repository docs/tests accordingly.
Changes:
- Introduces
CalendarService+CalendarEventDtoand a Riverpod provider to fetch calendar events via the portal session. - Exposes
portalDiofromPortalServiceso other services on the same host can reuse the authenticated client. - Adds integration/debug tests and updates
AGENTS.mdstatus entries.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
lib/services/calendar_service.dart |
New service/DTO/provider to fetch and parse calendar events from the portal JSON API. |
lib/services/portal_service.dart |
Exposes the underlying portal Dio instance for reuse by same-host services. |
test/services/calendar_service_test.dart |
Adds integration tests for calendar retrieval and basic DTO structure. |
test/services/cal_debug_test.dart |
Adds a manual-style debug test that prints raw/expanded calendar data. |
AGENTS.md |
Updates agent/status documentation to include CalendarService and a new “Last updated” date. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
rileychh
requested changes
Mar 5, 2026
…vice and remove CalendarService.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview
This pull request adds a new
CalendarServiceto the app, enabling retrieval of NTUT calendar events via the portal'scalModeApp.doJSON API. The service is integrated with the existing dependency injection setup and is thoroughly tested with new unit and debug tests. Documentation and agent status are updated to reflect this new capability.Academic Calendar Integration
CalendarServiceto fetch calendar events using the portal session andcalModeApp.doAPI. Includes a strongly-typed DTO for events and a Riverpod provider for DI.portalDioinPortalServicefor use by services sharing the portal host, such asCalendarService.Testing
CalendarServiceintest/services/calendar_service_test.dart, verifying event retrieval and structure.test/services/cal_debug_test.dartto print raw calendar event data for inspection.